fix: keep OpenJS Foundation untranslated per project policy#8848
fix: keep OpenJS Foundation untranslated per project policy#88487487 wants to merge 1 commit intonodejs:mainfrom
Conversation
- fr.json: revert "Fondation OpenJS" back to "OpenJS Foundation" - ta.json: revert Tamil translation back to "OpenJS Foundation" Per previous PR nodejs#6847, OpenJS Foundation is a proper noun and should not be translated in any locale. Co-Authored-By: Claude Opus 4.7 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit c64a844. Bugbot is set up for automated code reviews on this repo. Configure here. |
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website Please review the changes when you have a chance. Thank you! 🙏 |
|
Note Your Pull Request seems to be updating Translations of the Node.js Website. Whilst we appreciate your intent; Any Translation update should be done through our Crowdin Project. Thank you! |
There was a problem hiding this comment.
Pull request overview
Ensures the proper noun “OpenJS Foundation” remains untranslated in localized footer link text, aligning i18n strings with the project’s trademark/policy guidance referenced in prior work (PR #6847).
Changes:
- Update French footer link label to use “OpenJS Foundation”.
- Update Tamil footer link label to use “OpenJS Foundation”.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/i18n/src/locales/fr.json | Resets footer.links.foundationName from a French translation back to “OpenJS Foundation”. |
| packages/i18n/src/locales/ta.json | Resets footer.links.foundationName from a Tamil translation back to “OpenJS Foundation”. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -4,7 +4,7 @@ | |||
| "footer": { | |||
| "legal": "பதிப்புரிமை <foundationName>OpenJS நிறுவனம்</foundationName> மற்றும் Node.js பங்களிப்பாளர்கள். அனைத்து உரிமைகளும் பாதுகாக்கப்பட்டவை. <foundationName>OpenJS நிறுவனம்</foundationName> பதிவுசெய்யப்பட்ட வர்த்தக முத்திரைகளைக் கொண்டுள்ளது மற்றும் வர்த்தக முத்திரைகளைப் பயன்படுத்துகிறது. <foundationName>OpenJS நிறுவனம்</foundationName>-ன் வர்த்தக முத்திரைகளின் பட்டியலுக்கு, தயவுசெய்து எங்களது <trademarkPolicy>வர்த்தக முத்திரை கொள்கை</trademarkPolicy> மற்றும் <trademarkList>வர்த்தக முத்திரை பட்டியல் </trademarkList>-ஐப் பார்க்கவும்.<trademarkList>OpenJS Foundation வர்த்தக முத்திரைகளின் பட்டியலில்</trademarkList> குறிப்பிடப்படாத வர்த்தக முத்திரைகள் மற்றும் லோகோக்கள், அந்தந்த உரிமையாளர்களின் வர்த்தக முத்திரைகள்™ அல்லது பதிவுசெய்யப்பட்ட® வர்த்தக முத்திரைகள் ஆகும். அவற்றைப் பயன்படுத்துவது அவர்களுடனான எந்தவொரு தொடர்பையோ அல்லது அவர்களது அங்கீகாரத்தையோ குறிக்காது.", | |||
There was a problem hiding this comment.
The Tamil footer legal string still contains translated text inside <foundationName>…</foundationName> ("OpenJS நிறுவனம்"). If project policy is to keep “OpenJS Foundation” untranslated, these occurrences in legal should also be changed to “OpenJS Foundation”; updating links.foundationName alone won’t affect the rendered legal notice text.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c64a844. Configure here.
| "legal": "பதிப்புரிமை <foundationName>OpenJS நிறுவனம்</foundationName> மற்றும் Node.js பங்களிப்பாளர்கள். அனைத்து உரிமைகளும் பாதுகாக்கப்பட்டவை. <foundationName>OpenJS நிறுவனம்</foundationName> பதிவுசெய்யப்பட்ட வர்த்தக முத்திரைகளைக் கொண்டுள்ளது மற்றும் வர்த்தக முத்திரைகளைப் பயன்படுத்துகிறது. <foundationName>OpenJS நிறுவனம்</foundationName>-ன் வர்த்தக முத்திரைகளின் பட்டியலுக்கு, தயவுசெய்து எங்களது <trademarkPolicy>வர்த்தக முத்திரை கொள்கை</trademarkPolicy> மற்றும் <trademarkList>வர்த்தக முத்திரை பட்டியல் </trademarkList>-ஐப் பார்க்கவும்.<trademarkList>OpenJS Foundation வர்த்தக முத்திரைகளின் பட்டியலில்</trademarkList> குறிப்பிடப்படாத வர்த்தக முத்திரைகள் மற்றும் லோகோக்கள், அந்தந்த உரிமையாளர்களின் வர்த்தக முத்திரைகள்™ அல்லது பதிவுசெய்யப்பட்ட® வர்த்தக முத்திரைகள் ஆகும். அவற்றைப் பயன்படுத்துவது அவர்களுடனான எந்தவொரு தொடர்பையோ அல்லது அவர்களது அங்கீகாரத்தையோ குறிக்காது.", | ||
| "links": { | ||
| "foundationName": "OpenJS நிறுவனம்", | ||
| "foundationName": "OpenJS Foundation", |
There was a problem hiding this comment.
Tamil legal text still contains translated foundation name
Medium Severity
The fix updates links.foundationName to "OpenJS Foundation" but misses the legal field in ta.json, which still contains <foundationName>OpenJS நிறுவனம்</foundationName> (Tamil translation) in three places. The withLegal.tsx component renders text between <foundationName> tags as visible link text via t.rich(), so users will still see the translated name "OpenJS நிறுவனம்" in the legal paragraph, contradicting the policy that "OpenJS Foundation" remain untranslated everywhere.
Reviewed by Cursor Bugbot for commit c64a844. Configure here.
Description
Per previous PR #6847, "OpenJS Foundation" is a proper noun and should not be translated.
Two locale files incorrectly override it:
Changes
packages/i18n/src/locales/fr.json: Reset foundationNamepackages/i18n/src/locales/ta.json: Reset foundationNameCo-Authored-By: Claude Opus 4.7 [email protected]